Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Shipping network api #300

Merged
merged 6 commits into from
Mar 22, 2023
Merged

Shipping network api #300

merged 6 commits into from
Mar 22, 2023

Conversation

jumeyermattosarruda
Copy link
Contributor

What is the purpose of this pull request?

Adding shipping network endpoints to navigation.

How should this be manually tested?

Using our preview.

Screenshots or example usage

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

@netlify
Copy link

netlify bot commented Mar 22, 2023 •
edited
Loading

✅ Deploy Preview for elated-hoover-5c29bf ready!

Name Link
🔨 Latest commit 96a35f7
🔍 Latest deploy log https://app.netlify.com/sites/elated-hoover-5c29bf/deploys/641b62bb4c4a7a0008a6dbb8
😎 Deploy Preview https://deploy-preview-300--elated-hoover-5c29bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jumeyermattosarruda jumeyermattosarruda added the release-no No release bump label Mar 22, 2023
@jumeyermattosarruda
Copy link
Contributor Author

Mesmo fazendo as atualizações necessárias, o preview tá dando 404 😢

@brunoamui
Copy link
Collaborator

https://deploy-preview-300--elated-hoover-5c29bf.netlify.app/docs/api-reference/vtex-shipping-network-api tá funcionando sim. Só o navigation q no preview aparece o antigo. Testando com o script da @carolinamenezes é melhor.

@jumeyermattosarruda
Copy link
Contributor Author

Deu bom o preview do netlify 🤩🤩🤩🤩🤩

@jumeyermattosarruda jumeyermattosarruda merged commit e5aa078 into main Mar 22, 2023
@jumeyermattosarruda jumeyermattosarruda deleted the shipping-network-API branch March 22, 2023 20:39
@jumeyermattosarruda jumeyermattosarruda removed the request for review from carolinamenezes March 22, 2023 20:39
@jumeyermattosarruda jumeyermattosarruda restored the shipping-network-API branch March 22, 2023 21:23
@carolinamenezes carolinamenezes deleted the shipping-network-API branch January 12, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

@brunoamui brunoamui brunoamui approved these changes

Assignees
No one assigned
Labels
release-no No release bump
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jumeyermattosarruda @brunoamui